Peer Review Checklist: Embedded C Code 

Before Review: 

0 _Code compiles clean with extensive warning checks (e.g. MISRA C rules) 

Reviewer #1: 

1 _Commenting: top of file, start of function, code that needs an explanation 

2 _Style is consistent and follows style guidelines 

3 _ Proper modularity, module size, use of .h files and #includes 

4 _ No orphans (redundant, dead, commented out, unused code & variables) 

5 _Conditional expressions evaluate to a boolean value; no assignments 

6 _Parentheses used to avoid operator precedence confusion 

7 _All switch statements have a default clause; preferably an error trap 

Reviewer #2: 

8 _Single point of exit from each function 

9 _ Loop entry and exit conditions correct; minimum continue/break complexity 

10 _Conditionals should be minimally nested (generally only one or two deep) 

11 _All functions can be unit tested; SCC or SF complexity less than 10 to 15 

12 _ Use const and inline instead of #define; minimize conditional compilation 

13 _Avoid use of magic numbers (constant values embedded in code) 

14 _ Use strong typing (includes: sized types, structs for coupled data, const) 

15 _Variables have well chosen names and are initialized at definition 

Reviewer #3: 

16 _ Minimum scope for all functions and variables; essentially no globals 

17 _Concurrency issues? (locking, volatile keyword, minimize blocking time) 

18 _ Input parameter checking is done (style, completeness) 

19 _ Error handling for function returns is appropriate 

20 _ Null pointers, division by zero, null strings, boundary conditions handled 

21 _ Floating point use is OK (equality, NaN, INF, roundoff); use of fixed point 

22 _ Buffer overflow safety (bound checking, avoid unsafe string operations) 

All Reviewers 

23 _Does the code match the detailed design (correct functionality)? 

24 _Is the code as simple, obvious, and easy to review as possible? 

For TWO Reviewers assign items: ReviewerMl: 1-11; 23-24 ReviewerttZ: 12-24 
Items that are covered with static analysis can be removed from checklist 
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